Многопоточная обработка #14
Conversation
| @@ -164,6 +285,10 @@ func (s *databaseStorage) AddURLs(longURLs batchURLs, user string) (batchURLs, e | |||
|
|
|||
| func (s *databaseStorage) CloseFunc() func() { | |||
There was a problem hiding this comment.
есть ли необходимость в таком усложнении?
может стоит переименовать в Shutdown и явно вызывать при завершении работы сервиса?
There was a problem hiding this comment.
Это сделано для того, чтобы можно было универсально вернуть функцию в пакет main, вне зависимости от того, работаем мы с файлом или с БД. Проще повесить её на defer в основной программе, чем отслеживать факт завершения работы сервиса какими-либо специальными средствами.
There was a problem hiding this comment.
defer storage.Shutdown()
или определить анонимную функции в самом main
в рамках метода Shutdown оставить текущую логику, определенную в реализациях
| return urls | ||
| } | ||
|
|
||
| func (s *dummyStorage) deletionQueueProcess(ctx context.Context) { |
There was a problem hiding this comment.
метод с нереализованной логикой
есть ли необходимость?
| switch { | ||
| case database != "": | ||
| dStorage := newDBStorage(ctx, newMemoryStorage(), database) | ||
| dStorage.deletionCancel = deletionCancel |
No description provided.